Skip to content

ARROW-10364: [Dev][Archery] Add support for semver 2.13.0#8501

Closed
kou wants to merge 3 commits into
apache:masterfrom
kou:archery-semver-2.13.0
Closed

ARROW-10364: [Dev][Archery] Add support for semver 2.13.0#8501
kou wants to merge 3 commits into
apache:masterfrom
kou:archery-semver-2.13.0

Conversation

@kou

@kou kou commented Oct 21, 2020

Copy link
Copy Markdown
Member

No description provided.

@kou kou force-pushed the archery-semver-2.13.0 branch from 0303f34 to 7d1e824 Compare October 21, 2020 20:34
@github-actions

Copy link
Copy Markdown

@kszucs

kszucs commented Oct 22, 2020

Copy link
Copy Markdown
Member

I didn't see it before applied #8506 which also fixes the issue.

@kou

kou commented Oct 23, 2020

Copy link
Copy Markdown
Member Author

Which approach do you like?
Should we close this?

@terencehonles terencehonles left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Which approach do you like?
Should we close this?

I'll let you both work it out, but just calling out that they should work in tandem and this would fix any potential future string comparisons that are made that aren't covered with the test suite.

Comment thread dev/archery/archery/release.py Outdated
@emkornfield

Copy link
Copy Markdown
Contributor

@kou @kszucs any resolution on whether we want to merge this?

@kou kou force-pushed the archery-semver-2.13.0 branch from 7d1e824 to d6f3258 Compare October 30, 2020 02:05
@kou

kou commented Oct 30, 2020

Copy link
Copy Markdown
Member Author

@kszucs Can I merge this?

@pitrou

pitrou commented Nov 9, 2020

Copy link
Copy Markdown
Member

Ping @kszucs

@pitrou

pitrou commented Mar 30, 2021

Copy link
Copy Markdown
Member

Merging, we'll see if anyone complains :-)

@kou kou deleted the archery-semver-2.13.0 branch March 30, 2021 10:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants